Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added ability to unzip rar and 7z formats. #612

Closed
wants to merge 1 commit into from

Conversation

tkunstek
Copy link

Added ability to unzip rar and 7z formats. Now also finds files in subdirectories of all archive types.

@JoeMatt
Copy link
Member

JoeMatt commented Dec 7, 2017

In #566 I already added 7z through LzmaSDKObjC framework. It is missing RAR. I'd merge but there's a bunch of conflicts since I already added code paths for 7z.

@JoeMatt JoeMatt added the enhancement improvements, enhancements, new features, additions label Dec 17, 2017
@jasarien
Copy link
Member

@tkunstek would it be possible to remove the 7z support from this PR so that it only includes RAR? Or are the two formats provided by the same lib?

@tkunstek
Copy link
Author

Library used supports both formats. You can reject this PR and I'll come back through and add 7z separately.

@jasarien
Copy link
Member

jasarien commented Dec 20, 2017

Just to clarify it’s the RAR support that needs adding separately - 7z was added with @JoeMatt’s megaPR

@tkunstek
Copy link
Author

Ah. I can still add RAR too if it will benefit someone.

@tkunstek tkunstek closed this Dec 20, 2017
@JoeMatt
Copy link
Member

JoeMatt commented Dec 20, 2017

If this were to be added it would make sense to remove my LzmaSDKObjC dependency and depend on SARUnArchiveANY solely to reduce dependency management.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improvements, enhancements, new features, additions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants